Skip to content
This repository has been archived by the owner on Mar 21, 2024. It is now read-only.

Fixes Issues #161 and #162 #163

Merged
merged 2 commits into from
Jul 30, 2021
Merged

Conversation

seunghwak
Copy link
Contributor

Close #161 Close #162

@alliepiper alliepiper reopened this Jul 30, 2021
@alliepiper alliepiper self-assigned this Jul 30, 2021
@alliepiper alliepiper added the type: bug: functional Does not work as intended. label Jul 30, 2021
@alliepiper alliepiper added this to the 1.14.0 milestone Jul 30, 2021
@alliepiper
Copy link
Collaborator

I ran into these issues while adding tests for DeviceSpmv. Looks like this PR was accidentally closed during the repo reorg last year.

Reopening.

@alliepiper alliepiper changed the base branch from 1.8.0 to main July 30, 2021 03:22
@@ -664,12 +664,12 @@ struct DispatchSpmv

// Get grid dimensions
dim3 spmv_grid_size(
CUB_MIN(num_merge_tiles, max_dim_x),
CUB_MIN(num_merge_tiles, static_cast<unsigned int>(max_dim_x)),
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These were fixed earlier by b2e64cf. I'll drop this while rebasing.

@alliepiper alliepiper force-pushed the bug_ext_memory_pool branch from 3749499 to 01980f9 Compare July 30, 2021 03:32
alliepiper added a commit to alliepiper/thrust that referenced this pull request Jul 30, 2021
@alliepiper
Copy link
Collaborator

DVS CL: 30244015
gpuCI: NVIDIA/thrust#1499

@alliepiper alliepiper added testing: gpuCI in progress Started gpuCI testing. testing: internal ci in progress Currently testing on internal NVIDIA CI (DVS). labels Jul 30, 2021
@alliepiper alliepiper linked an issue Jul 30, 2021 that may be closed by this pull request
alliepiper added a commit to alliepiper/cub that referenced this pull request Jul 30, 2021
@alliepiper alliepiper added testing: gpuCI passed Passed gpuCI testing. testing: internal ci passed Passed internal NVIDIA CI (DVS). and removed testing: gpuCI in progress Started gpuCI testing. testing: internal ci in progress Currently testing on internal NVIDIA CI (DVS). labels Jul 30, 2021
@alliepiper
Copy link
Collaborator

Looks good -- thanks for the patch, sorry it took so long to merge it!

@alliepiper alliepiper merged commit 9fffdf3 into NVIDIA:main Jul 30, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
testing: gpuCI passed Passed gpuCI testing. testing: internal ci passed Passed internal NVIDIA CI (DVS). type: bug: functional Does not work as intended.
Projects
None yet
3 participants